Skip to content

Conversation

@resistor
Copy link
Collaborator

@resistor resistor commented Jun 5, 2025

No description provided.

arichardson and others added 9 commits June 5, 2025 16:12
These add unnecessary noise when running tests.
This test just checks that we don't crash and has no further information.
Back in 2018 adding custom type checking to the intrinsic would have
added a larger downstream diff, but since 8392f1c it has the
custom type checking so we can overload it for capabilities.
The header link.h might not exist for baremetal targets. Only include it
when we actually have the c18n support.
Can't use the MTE inline assembly when building for purecap Morello
This may have been needed before ISAInfo handled custom extensions, but
now we can just defer to ISAInfo. No functional change intended
The upcoming standard no longer includes cgetoffset (and cgetaddr has
been removed in ISAv9). For now the assembly code still accepts cgetaddr
and converts it to a move, but use an intrinsic call instead as that can
be optimized to a subregister read on next use rather the requiring an
explicit move.
This also handles implied features and therefore is more future-proof than
a simple string comparison.
We have to use XLenVT for getting the intrinsic type otherwise we end
up triggering an assertion in LegalizeDAG.
@resistor resistor merged commit 33cdae8 into CHERIoT-Platform:cheriot Jun 6, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants